[IMP] l10n_ar_account_reports: checks de pagos en borrador#869
[IMP] l10n_ar_account_reports: checks de pagos en borrador#869pablohmontenegro wants to merge 1 commit intoingadhoc:19.0from
Conversation
|
|
There was a problem hiding this comment.
Pull request overview
Este PR añade un check específico para detectar pagos de retenciones de ganancias en estado borrador en el reporte SICORE, mejorando la validación de datos antes de enviar el reporte.
Cambios clave:
- Agrega validación para pagos de retención en estado borrador dentro del período del reporte SICORE
- Implementa un nuevo check que alerta sobre retenciones de ganancias no confirmadas
| @@ -217,6 +217,37 @@ def _run_checks(self, check_codes_to_ignore): | |||
| # smplificamos check de todos los reportes argentinos | |||
There was a problem hiding this comment.
Error de tipeo: "smplificamos" debería ser "simplificamos".
| # smplificamos check de todos los reportes argentinos | |
| # simplificamos check de todos los reportes argentinos |
| if self.company_id.country_id.code == "AR" and self.is_tax_return: | ||
| # por ahora ignoramos todos los checks nativos para simplificar | ||
| if self.type_external_id == "l10n_ar_account_reports.sicore_return_type": | ||
| checks = [] |
There was a problem hiding this comment.
La variable checks se inicializa en la línea 221 pero nunca se usa. Debería eliminarse esta inicialización ya que se redefine completamente en la línea 249 con checks = [check_vals].
| checks = [] |
| "check_company_data", | ||
| ] | ||
| ) | ||
| return super()._run_checks(check_codes_to_ignore) |
There was a problem hiding this comment.
Se agrega un segundo return super()._run_checks(check_codes_to_ignore) en la línea 260, cuando ya existe uno en la línea 261. Esto genera código muerto: la línea 261 nunca se alcanzará porque la línea 260 ya retorna. Debes eliminar una de estas dos líneas. Dado que la línea 260 es la nueva, considera si la intención era reemplazar el return existente o agregar lógica adicional antes del return.
| return super()._run_checks(check_codes_to_ignore) |
pablohmontenegro
force-pushed
the
19.0-t-56583-pam
branch
5 times, most recently
from
4834a4d to
0b05630
Compare
pablohmontenegro
changed the title
pablohmontenegro
force-pushed
the
19.0-t-56583-pam
branch
from
0b05630 to
5b10176
Compare
1) Mejorar check de pagos en borrador 2) Cuenta opcional en el return type
pablohmontenegro
force-pushed
the
19.0-t-56583-pam
branch
from
5b10176 to
5c1f045
Compare
Uh oh!
There was an error while loading. Please reload this page.